tool(ws): Switch ws runner to <workspace>:<script> syntax#797
Open
necolas wants to merge 1 commit into
Open
Conversation
Change the workspace script runner from a space-separated `bun ws <workspace> <script>` form to a single `<workspace>:<script>` target token. The package part is everything before the first colon, so script names that contain colons (e.g. `docs:trees:dev`) stay intact. An empty package part fans out to every workspace, so `bun ws :test` runs `test` in every workspace that defines it, equivalent to `'*:test'`. Globs in the package part are still supported (`'packages/*:build'`). Fan-out only runs the script where it exists, which is bun's existing `-F` behavior; a single explicit workspace still errors on a missing script. The old space-separated form now prints a helpful error pointing at the new syntax. Update all call sites and docs: root and CI scripts, app `build:deps`, package READMEs, benchmark/profile usage strings, and the profile CLI test assertion.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
necolas
requested review from
SlexAxton and
amadeus
and removed request for
amadeus
SlexAxton
approved these changes
Jun 10, 2026
amadeus
approved these changes
Jun 10, 2026
amadeus
left a comment
amadeus
left a comment
Member
There was a problem hiding this comment.
lgtm. Could we maybe have a shortcut for building all packages?
Contributor
|
bun ws :build
…On Wed, Jun 10, 2026 at 4:28 PM Amadeus Demarzi ***@***.***> wrote:
***@***.**** approved this pull request.
lgtm. Could we maybe have a shortcut for building all packages?
—
Reply to this email directly, view it on GitHub
<#797?email_source=notifications&email_token=AAAXSKSXEVN5QLE67FHNDWD47HHJDA5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTINBXGE3TAOBQGAZKM4TFMFZW63VQOJSXM2LFO5PXEZLROVSXG5DFMSSWK5TFNZ2KYZTPN52GK4S7MNWGSY3L#pullrequestreview-4471708002>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAXSKSOKH2AF2GJ5LS3UZ347HHJDAVCNFSNUABGKJSXA33TNF2G64TZHMYTANRQGMYDANZXGM5US43TOVSTWNBWGI3TCMZVGIZDJILWAI>
.
Triage notifications, keep track of coding agent tasks and review pull
requests on the go with GitHub Mobile for iOS
<https://github.com/notifications/mobile/ios/AAAXSKXPCMQIHM3EHO235TD47HHJDA5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTINBXGE3TAOBQGAZKM4TFMFZW63VQOJSXM2LFO5PXEZLROVSXG5DFMSSWK5TFNZ2KUZTPN52GK4S7NFXXG>
and Android
<https://github.com/notifications/mobile/android/AAAXSKWDJK43WPEG37QEGW347HHJDA5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTINBXGE3TAOBQGAZKM4TFMFZW63VQOJSXM2LFO5PXEZLROVSXG5DFMSSWK5TFNZ2K4ZTPN52GK4S7MFXGI4TPNFSA>.
Download it today!
You are receiving this because your review was requested.Message ID:
***@***.***>
|
Member
|
@SlexAxton does that build apps too? or is that scoped to only packages? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Changes the workspace script runner from the space-separated
bun ws <workspace> <script>form to a single<workspace>:<script>target token.Behavior
bun ws docs:trees:dev→ workspacedocs, scripttrees:dev).:test) fans out to every workspace, equivalent to'*:test'.-Fbehavior). A single explicit workspace still errors on a missing script.Call sites updated
Root
tscscript, CI workflow, appbuild:deps, package READMEs, benchmark/profile usage strings,AGENTS.md, and the profile CLI test assertion.Verification
bun run formatclean;bun run lintshows only pre-existing findings in untouched files.packages/trees/test/file-tree-profile-cli.test.tspasses (asserts the new usage string).bun ws :testconfirmed to fan out only to packages with atestscript; error paths (old syntax, missing script, missing workspace, empty script) all verified.